-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Access] Response limit tracker #6814
[Access] Response limit tracker #6814
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6814 +/- ##
==========================================
- Coverage 41.18% 41.18% -0.01%
==========================================
Files 2109 2109
Lines 185660 185664 +4
==========================================
Hits 76467 76467
- Misses 102782 102786 +4
Partials 6411 6411
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@Guitarheroua please address merge conflicts |
…isarchuk/6640-responce-limit-tracker
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Left a few comments tests related.
Co-authored-by: Yurii Oleksyshyn <[email protected]>
Closes: #6640
NOTE: this PR should be merged after #6798
Implements rate limiter for responses, that controller returns to client.